-
Notifications
You must be signed in to change notification settings - Fork 75
Add Remix IDE (new IA) #1174
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Remix IDE (new IA) #1174
Conversation
β¦Contract' section in navigation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the Remix IDE documentation to reflect the migration from Polkadot-specific Remix IDE (remix.polkadot.io) to the standard Ethereum Remix IDE (remix.ethereum.org) for use with Polkadot Hub. The changes update references, wallet providers, and add comprehensive troubleshooting documentation.
- Migrated from Polkadot Remix IDE to standard Ethereum Remix IDE
- Updated wallet provider from Talisman to MetaMask as the primary example
- Added comprehensive troubleshooting guide with common issues and solutions
Reviewed Changes
Copilot reviewed 4 out of 21 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| troubleshooting.md | Added new comprehensive troubleshooting guide covering compilation, wallet connection, deployment, contract interaction, and browser issues |
| get-started.md | Updated to reference remix.ethereum.org and MetaMask instead of Polkadot-specific IDE and Talisman; removed deployment/interaction sections (moved to separate file) |
| deploy-a-contract.md | Added new guide focused on contract deployment and interaction using wallet providers |
| .nav.yml | Commented out "Verify a Contract" navigation item with TODO note |
π‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automated style guide suggestions (1-15 of 15)
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Handled robot feedbacks. Once this gets tech review, I'll run through it one more time. |
brunopgalvao
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Left one comment.
|
I updated the formatting on the Troubleshooting page to use the FAQ interface style admonitions. You'll need to serve it with: papermoonio/polkadot-mkdocs#173 to see the correct formatting. |
dawnkelly09
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ready for a sanity check by Erin. Needs mkdocs PR: papermoonio/polkadot-mkdocs#173 for proper styling of troubleshooting/faq page
eshaben
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gahhhhhh I commented on the AI pages π€¦ββοΈ - no going back now haha, sorry
.ai/pages/smart-contracts-dev-environments-remix-get-started.md
Outdated
Show resolved
Hide resolved
.ai/pages/smart-contracts-dev-environments-remix-get-started.md
Outdated
Show resolved
Hide resolved
.ai/pages/smart-contracts-dev-environments-remix-get-started.md
Outdated
Show resolved
Hide resolved
.ai/pages/smart-contracts-dev-environments-remix-get-started.md
Outdated
Show resolved
Hide resolved
β¦ on Polkadot Hub.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll approve this now so we can get it in. But there are still some things we should fix at some point.
Edit: Ha just kidding - can't merge until the merge conflict issue I left a comment about is resolved
images/smart-contracts/dev-environments/remix/deploy-a-contract/remix-2.gif
Show resolved
Hide resolved
dawnkelly09
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
π’
π Description
Provide a clear and concise description of your changes.
π Review Preference
Choose one:
π€ AI-Ready Docs
If content changed, regenerate AI files:
python3 scripts/generate_llms.pyβ Checklist